-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Introduce accumulating statistic in time-series DB #325
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #325 +/- ##
==========================================
+ Coverage 79.06% 79.11% +0.05%
==========================================
Files 21 22 +1
Lines 1614 1656 +42
==========================================
+ Hits 1276 1310 +34
- Misses 338 346 +8 ☔ View full report in Codecov by Sentry. |
dbf3d63
to
065ff45
Compare
d07fc7f
to
a7f476d
Compare
b6dbc73
to
f9ad91d
Compare
d026e57
to
5f4ba7f
Compare
Idea is to start gathering statistic about amount of entities existing in every account at every point of time. This will allow detect different kind of anomalies in testing pipelines (e.g. bugs in scheduling , bugs in cleanup etc.) This PR just introduce functionality which allow to achieve this in future PRs we will start using it in EVERY query for EVERY cloud provider.
@ricardobranco777 - please check latest version , is that what you suggested ? |
This is pure beauty!!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel more comfortable now merging this as we're adding the new functionality without modifying existing code.
Nice work!
Idea is to start gathering statistic about amount of entities existing in every account at every point of time . This will allow detect different kind of anomalies in testing pipelines ( e.g. bugs in scheduling , bugs in cleanup etc. )
This PR just introduce functionality which allow to achieve this in future PRs we will start using it in EVERY query for EVERY cloud provider